Skip to content

add embedded scenario for meldemichel#214

Merged
warm-coolguy merged 6 commits intomainfrom
add-meldemichel-embedded-scenario
Jan 21, 2025
Merged

add embedded scenario for meldemichel#214
warm-coolguy merged 6 commits intomainfrom
add-meldemichel-embedded-scenario

Conversation

@warm-coolguy
Copy link
Member

Summary

An embedded scenario has been added to the Meldemichel. This pertains purely to example code and documentation since it was always possible to do this in POLAR, but not explicitly documented for the Meldemichel yet.

Instructions for local reproduction and review

Build the documentation pages and check the generated scenario on desktop/mobile. The links in the API.md will only start working after this PR is merged.

@warm-coolguy warm-coolguy added the documentation Improvements or additions to documentation label Jan 8, 2025
@warm-coolguy warm-coolguy self-assigned this Jan 8, 2025
Copy link
Member

@dopenguin dopenguin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just some minor suggestions

🏓 @warm-coolguy

@warm-coolguy warm-coolguy force-pushed the add-meldemichel-embedded-scenario branch from 31efb29 to b01b43f Compare January 20, 2025 05:57
warm-coolguy and others added 2 commits January 20, 2025 06:58
apply suggestion "fix example id"

Co-authored-by: Pascal Röhling <73653210+dopenguin@users.noreply.github.com>
apply suggestion "insist upon <noscript>"

Co-authored-by: Pascal Röhling <73653210+dopenguin@users.noreply.github.com>
@warm-coolguy
Copy link
Member Author

Just some minor suggestions

🏓 @warm-coolguy

Thanks, the id catch was great, that would've sparked quite some confusion.

🏓 @dopenguin

Copy link
Member

@dopenguin dopenguin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@warm-coolguy warm-coolguy merged commit 8dee461 into main Jan 21, 2025
4 checks passed
@warm-coolguy warm-coolguy deleted the add-meldemichel-embedded-scenario branch January 21, 2025 08:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants